Skip to content

docs: fix broken relative links on the SDK overview page#994

Merged
vdusek merged 2 commits into
masterfrom
docs/fix-overview-links
Jun 17, 2026
Merged

docs: fix broken relative links on the SDK overview page#994
vdusek merged 2 commits into
masterfrom
docs/fix-overview-links

Conversation

@vdusek

@vdusek vdusek commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

The overview page links to other docs with relative paths like ../concepts/.... Because the page sets slug: /overview, its URL is flattened to /sdk/python/docs/overview, so each ../ over-traverses and drops the /docs/ segment. The result is broken links such as /sdk/python/concepts/actor-lifecycle instead of /sdk/python/docs/concepts/actor-lifecycle.

This corrects the relative depth to match the flattened URL (./concepts/..., ./guides/...), keeping the path-based link style.

It also gives the quick start page its own slug: /quick-start, so it is served at /sdk/python/docs/quick-start instead of /sdk/python/docs/introduction/quick-start. The overview link goes back to ./quick-start, and the one cross-link from the v1.7 requests-and-httpx guide is updated to match.

Applied to the current docs and to all versioned snapshots (3.4, 2.7, 1.7, 0.2). The live page is served from the latest release (3.4), so it is fixed too.

@vdusek vdusek added adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team. labels Jun 17, 2026
@vdusek vdusek self-assigned this Jun 17, 2026
@github-actions github-actions Bot added this to the 143rd sprint - Tooling team milestone Jun 17, 2026
@codecov

codecov Bot commented Jun 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 90.54%. Comparing base (726620b) to head (84ffccb).
⚠️ Report is 2 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #994   +/-   ##
=======================================
  Coverage   90.54%   90.54%           
=======================================
  Files          49       49           
  Lines        3132     3132           
=======================================
  Hits         2836     2836           
  Misses        296      296           
Flag Coverage Δ
e2e 36.14% <ø> (ø)
integration 57.15% <ø> (ø)
unit 79.27% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@vdusek vdusek marked this pull request as ready for review June 17, 2026 09:34
@vdusek vdusek requested a review from szaganek as a code owner June 17, 2026 09:34
@vdusek vdusek merged commit 81b0110 into master Jun 17, 2026
28 checks passed
@vdusek vdusek deleted the docs/fix-overview-links branch June 17, 2026 09:38
vdusek added a commit that referenced this pull request Jun 17, 2026
…-start (#997)

Follow-up to #994, fixing the broken relative links it didn't cover.
Verified with a full `onBrokenLinks: 'throw'` docs build (0 broken
links).

- **Versioned reference links (1.7, 2.7):** `../../reference/...`
over-traversed because these versions are served under a
`/docs/<version>/` URL prefix. Corrected to `../../../reference/...`.
- **Quick-start outbound links (all versions):** #994's new `slug:
/quick-start` flattened these pages one level up, so their own
`../concepts/...` and `../guides/...` links over-traversed. Corrected to
`./concepts/...` / `./guides/...` (the same fix #994 applied to the
overview page).
- **Quick-start platform links (current + 3.4):** bare `/platform/...`
resolved to a non-existent path under the `/sdk/python` baseUrl. Changed
to absolute `https://docs.apify.com/platform/...`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

adhoc Ad-hoc unplanned task added during the sprint. t-tooling Issues with this label are in the ownership of the tooling team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants